Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix defines around fixed-mag-cal-yaw #26126

Merged
merged 4 commits into from Feb 6, 2024
Merged

Fix defines around fixed-mag-cal-yaw #26126

merged 4 commits into from Feb 6, 2024

Conversation

peterbarker
Copy link
Contributor

... also change the main method to return bool not MAV_RESULT.

... also add options to custom build server. Saves ~2kB when fixed-yaw is removed (this also removes in-flight learning).

Board               AP_Periph  antennatracker  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
Durandal                       8               8      *           16      16                24     8      8
HerePro             *                                                                                     
Hitec-Airspeed      *                                                                                     
KakuteH7-bdshot                16              16     *           16      16                16     16     16
MatekF405                      16              16     *           16      16                16     16     16
Pixhawk1-1M-bdshot             16              16                 16      16                16     16     16
f103-QiotekPeriph   *                                                                                     
f303-Universal      *                                                                                     
iomcu                                                                           *                         
revo-mini                      16              16     *           16      16                16     16     16
skyviper-v2450                                                    16

@peterbarker
AP_Compass: make compass.mag_cal_fixed_yaw return boolean
2801860 
this method could be used from a transmitter without GCS enabled, for example
@peterbarker
AP_HAL_ChibiOS: make compass.mag_cal_fixed_yaw return boolean
671411f 
this method could be used from a transmitter without GCS enabled, for example
@peterbarker
GCS_MAVLink: make compass.mag_cal_fixed_yaw return boolean
03753d6 
this method could be used from a transmitter without GCS enabled, for example
@tridge tridge merged commit 857547b into ArduPilot:master Feb 6, 2024
92 checks passed
@peterbarker peterbarker deleted the pr/fixed-yaw-cal-fixes branch February 6, 2024 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amilcarlucas amilcarlucas amilcarlucas left review comments

@tridge tridge tridge approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@peterbarker @tridge @amilcarlucas @CraigElder